Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: update README about start server with example graph #2315

Merged
merged 3 commits into from
Oct 2, 2023

Conversation

aroundabout
Copy link
Contributor

Purpose of the PR

  • update readme about start server with example graph

Main Changes

  • update readme about start server with example graph
  • update the link of the subproject of toolchain

Verifying these changes

  • Trivial rework / code cleanup without any test coverage. (No Need)
  • Already covered by existing tests, such as (please modify tests here).
  • Need tests and can be verified as follows:
    • xxx

Does this PR potentially affect the following parts?

  • Nope
  • Dependencies (add/update license info)
  • Modify configurations
  • The public API
  • Other affects (typed here)

Documentation Status

  • Doc - TODO
  • Doc - Done
  • Doc - No Need

imbajin

This comment was marked as off-topic.

@imbajin imbajin changed the title doc: update README about start server with example graph doc(hubble): update README about start server with example graph Sep 19, 2023
@imbajin imbajin changed the title doc(hubble): update README about start server with example graph doc: update README about start server with example graph Sep 19, 2023
@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Merging #2315 (44ed87f) into master (4ceef1a) will decrease coverage by 3.44%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2315      +/-   ##
============================================
- Coverage     68.26%   64.83%   -3.44%     
+ Complexity      989      987       -2     
============================================
  Files           500      500              
  Lines         41516    41516              
  Branches       5786     5786              
============================================
- Hits          28342    26915    -1427     
- Misses        10415    11917    +1502     
+ Partials       2759     2684      -75     

see 71 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

README.md Outdated Show resolved Hide resolved
Co-authored-by: imbajin <[email protected]>
javeme
javeme previously approved these changes Sep 30, 2023
README.md Outdated
@@ -54,7 +57,7 @@ The project [doc page](https://hugegraph.apache.org/docs/) contains more informa
and provides detailed documentation for users. (Structure / Usage / API / Configs...)

And here are links of other **HugeGraph** component/repositories:
1. [hugegraph-toolchain](https://github.com/apache/incubator-hugegraph-toolchain) (graph **loader/dashboard/tool/client**)
1. [hugegraph-toolchain](https://github.com/apache/incubator-hugegraph-toolchain) (graph **[loader](https://github.com/apache/incubator-hugegraph-toolchain/tree/master/hugegraph-client)/[dashboard](https://github.com/apache/incubator-hugegraph-toolchain/tree/master/hugegraph-hubble)/[tool](https://github.com/apache/incubator-hugegraph-toolchain/tree/master/hugegraph-tools)/[client](https://github.com/apache/incubator-hugegraph-toolchain/tree/master/hugegraph-client)**)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

loader link seems not right?

README.md Outdated Show resolved Hide resolved
@javeme javeme merged commit d7c1c21 into apache:master Oct 2, 2023
17 of 21 checks passed
VGalaxies pushed a commit to VGalaxies/incubator-hugegraph that referenced this pull request Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants